-
Notifications
You must be signed in to change notification settings - Fork 1
Feature/188 쪽지 상세 보기 기능 구현 #193
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- 쿼리 메서드 생성 - mapper record 클래 생성 - 테스트 코드 작성및 검증 완료
- 쿼리 메서드 생성 - mapper record 클래스 생성 - 테스트 코드 작성및 검증 완료
- NoSuchElementException 클래스 추가 - 예외메세지 추가 - 전역 예외처리 핸들러에 NoSuchElementException 추가
- ExceptionMessage enum 값의 내부 메세지가 아닌 ExceptionMessage enum 자체를 이용하여 생성하도록 수정
- 기관의 쪽지 상세보기 메서드 구현 - 봉사자의 쪽지 상세보기 메서드 구현 - 테스트 코드 작성및 검증 완료
- 컨트롤러 구현 - 테스트 코드 작성및 검증 완료
- 날짜 비교 검증 수
m-a-king
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다~
| } | ||
|
|
||
| @Secured("ROLE_VOLUNTEER") | ||
| @Operation(summary = "기관의 자신에게 온 쪽지 상세 조회") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
봉사자가 맞는 것 같아요~
| note.content, | ||
| volunteer.id.as(SENDER_ID), | ||
| volunteer.nickname.as(SENDER_NAME), | ||
| volunteer.imgUrl.as("senderProfileImgLink"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이것도 상수로 처리할 수 있을 것 같아요
leebs0521
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다. 재중님 말씀한것만 고치면 되겠네요.
- 컨트롤러 swagger 설명 수정 - 매직 스트링의 상수화
|



📌 과제 설명
기관/봉사자의 자신에게 온 쪽지 상세 조회 기능입니다.
👩💻 요구 사항과 구현 내용
✅ PR 포인트 & 궁금한 점